Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Run/Install instructions + AUR Package #149

Closed
wants to merge 1 commit into from

Conversation

aakashhemadri
Copy link

Signed-off-by: Aakash Hemadri [email protected]

Signed-off-by: Aakash Hemadri <[email protected]>
@aakashhemadri
Copy link
Author

@pixeltris I wanted to support this port on AUR I thought it'd be nice if I added the information to the README. Open to changes :)

@pixeltris
Copy link
Owner

The README needs improvements in general as mentioned by a few people that instructions aren't super clear.

I'm going to hold off on merging this and look at overhauling the README some-when soon, which would include the info about your package.

@konsumer
Copy link

I missed this PR, and did one for Linux, too (#162) Sorry. Feel free to remove it, or merge them all together however you want.

@aakashhemadri
Copy link
Author

@konsumer The udev rule section is helpful! Thanks for the nice contribution. @pixeltris Would be great if we can update the README, If possible can you express your expectations in the form of sub headings. I don't mind filling in the details.

@konsumer
Copy link

I missed an s on the rules filename, but added that to the other PR. should be /etc/udev/rules.d/gk6x.rules

@pixeltris
Copy link
Owner

@aakashhemadri I've gone ahead and made the README changes I mentioned I was going to make. I've included your AUR package under the System requirements / compilation section. It's currently under a drop down as the README is a little noisy with the more verbose instruction. I hope this suffices.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants